Skip to content

[SC-13093] Fix plotly express figures not displaying in UI#447

Merged
juanmleng merged 3 commits intomainfrom
juan/sc-13093/fix-plotly-express-figures-not-displaying-in-ui
Nov 11, 2025
Merged

[SC-13093] Fix plotly express figures not displaying in UI#447
juanmleng merged 3 commits intomainfrom
juan/sc-13093/fix-plotly-express-figures-not-displaying-in-ui

Conversation

@juanmleng
Copy link
Contributor

Pull Request Description

What and why?

  • Capped plotly version to <=5.24.1 in pyproject.toml
  • Plotly Express introduced binary data encoding ("bdata") in plotly 6.4.0+ when serializing figures. The frontend cannot decode this binary format, causing figures created with px.histogram(), px.box(), and px.density_contour() to not display in the UI, even though they were correctly generated and saved to S3.

How to test

Run rag_documentation_demo.ipynb shows figures from embeddings and rag tests correctly.

What needs special review?

Dependencies, breaking changes, and deployment notes

Release notes

Checklist

  • What and why
  • Screenshots or videos (Frontend)
  • How to test
  • What needs special review
  • Dependencies, breaking changes, and deployment notes
  • Labels applied
  • PR linked to Shortcut
  • Unit tests added (Backend)
  • Tested locally
  • Documentation updated (if required)
  • Environment variable additions/changes documented (if required)

@juanmleng juanmleng self-assigned this Nov 10, 2025
@juanmleng juanmleng added bug Something isn't working internal Not to be externalized in the release notes labels Nov 10, 2025
Copy link
Contributor

@johnwalz97 johnwalz97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one comment on the version spec but lgtm

@github-actions
Copy link
Contributor

PR Summary

This PR makes several dependency and version updates in the lock file and project metadata. The key changes are:

  1. The lock file (poetry.lock) has been regenerated using Poetry 2.1.3. Several new wheel entries have been added for the Brotli package, targeting additional architectures and musllinux variants, which should improve support on various platforms.

  2. The narwhals package entry has been removed from the lock file along with its extras. This change removes the dependency on a lightweight compatibility layer between dataframe libraries.

  3. The Plotly package has been downgraded from version 6.3.0 to 5.24.1. The change includes updates to the package description and associated file hashes. In addition, the dependency constraint in pyproject.toml has been adjusted to "plotly (>=5.0.0,<6.0.0)" to ensure compatibility.

  4. Finally, the project version has been bumped from 2.10.2 to 2.10.3 in pyproject.toml and in the version.py file.

These changes primarily affect dependency management and package distribution, and they do not introduce any new application logic.

Test Suggestions

  • Run the full test suite to ensure that changes in dependencies do not break existing functionality.
  • Verify that the application runs correctly on all supported platforms, especially where new Brotli wheels have been added.
  • Test installation from the generated lock file in a clean environment to ensure that dependency resolution works as expected.
  • Check for any issues due to the removal of the narwhals package if it was implicitly expected elsewhere in the project.

@juanmleng juanmleng merged commit 9c9acac into main Nov 11, 2025
17 checks passed
@juanmleng juanmleng deleted the juan/sc-13093/fix-plotly-express-figures-not-displaying-in-ui branch November 11, 2025 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working internal Not to be externalized in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants